Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STCLI-255 add stripes test cypress command #362

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from
Draft

Conversation

JohnC-80
Copy link
Contributor

@JohnC-80 JohnC-80 commented Nov 8, 2024

This PR adds the cypress sub-command to the test command..

Run cypress tests headlessly...

stripes test cypress

Run cypress tests in the cypress UI (handy for development)

stripes test cypress --cypress.open

Since cypress insists that repos contain their own cypress.config.js file, stripes-cli exports a function to generate a base configuration - it generates the defaults internally and can be used at the repo level to override the configuration. stripes-components PR for example use: https://github.com/folio-org/stripes-components/pull/2386/files#diff-5bae461d15457335f3d6285d35d3df2fd618e583eb7460c52cd8f0de69627abc

We'll have to determine if other 'cypress resources' are necessary and automatically added when cypress is implemented via this tooling - will have to try it with nothing...

This code adds a first draft for a standalone function that generates a webpack configuration without requiring a previous setup of a platform or "StripesCore" (ahem stripes-webpack.)

Remaining work:

Documentation!
Add code coverage support!
Attempt to convert stripes-core and stripes-smart-components

Copy link

github-actions bot commented Nov 8, 2024

Jest Unit Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 127c779. ± Comparison against base commit 082c7db.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant